Skip to content

Add configurable confirm dialog logo and update Project Settings#635

Open
xkello wants to merge 2 commits into
developfrom
implement-#3345
Open

Add configurable confirm dialog logo and update Project Settings#635
xkello wants to merge 2 commits into
developfrom
implement-#3345

Conversation

@xkello
Copy link
Copy Markdown
Contributor

@xkello xkello commented Jun 1, 2026

Add configuration to change confirm dialog logo bg to fill the figma (https://www.figma.com/design/Zl8irhQtrHrxBVcJyLPedL/Dashboard-2.0?node-id=3355-10739&t=mtqPIiHeupfZI8d1-0) requirements. Behavior only applies when it is defined for the logo. When its left empty (default setting) the logo follows old logic - bg changes according to button variant.

Update some names according to figma specs above

@xkello xkello requested a review from MarcelGeo June 1, 2026 07:53
@coveralls
Copy link
Copy Markdown

coveralls commented Jun 1, 2026

Coverage Report for CI Build 26746454661

Coverage remained the same at 93.215%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 9830
Covered Lines: 9163
Line Coverage: 93.21%
Coverage Strength: 0.93 hits per line

💛 - Coveralls


const logoSrc = computed(() => {
const variant =
props.logoVariant === 'auto'
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what means auto here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thats the fall back logic. If not variant is specified use the default logic and use logo based on severity

Comment thread web-app/packages/lib/src/modules/project/views/ProjectViewTemplate.vue Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants